Skip to content

tests(nmrs): .ovpn parser tests and refactored logic in arms#316

Merged
cachebag merged 1 commit intodev-openvpnfrom
ovpn-parser
Apr 2, 2026
Merged

tests(nmrs): .ovpn parser tests and refactored logic in arms#316
cachebag merged 1 commit intodev-openvpnfrom
ovpn-parser

Conversation

@cachebag
Copy link
Copy Markdown
Owner

@cachebag cachebag commented Apr 1, 2026

This PR adds a suite of unit tests for the parsing logic and refactors some of the parsing logic to follow some rust idioms.

@cachebag cachebag requested a review from stoutes April 1, 2026 18:54
@cachebag cachebag self-assigned this Apr 1, 2026
@cachebag cachebag added nmrs Changes to nmrs refactor Change or improve code tests Add, remove or adjust tests labels Apr 1, 2026
@cachebag cachebag force-pushed the ovpn-parser branch 2 times, most recently from 3fec0bf to 78d866a Compare April 1, 2026 19:00
@cachebag
Copy link
Copy Markdown
Owner Author

cachebag commented Apr 1, 2026

Fuck you clippy we're in dev

SemVer check can also fail right now, we do not concern ourselves with API stability in a dev branch

@cachebag cachebag force-pushed the dev-openvpn branch 2 times, most recently from c1acdfe to 5d65aa0 Compare April 1, 2026 19:11
@cachebag cachebag merged commit 32a5cc7 into dev-openvpn Apr 2, 2026
4 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

nmrs Changes to nmrs refactor Change or improve code tests Add, remove or adjust tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant